Skip to content

refactor(config): replace IConfig with IAppConfig for improved type safety and clarity#3321

Open
Chartman123 wants to merge 1 commit intomainfrom
chore/appconfig
Open

refactor(config): replace IConfig with IAppConfig for improved type safety and clarity#3321
Chartman123 wants to merge 1 commit intomainfrom
chore/appconfig

Conversation

@Chartman123
Copy link
Copy Markdown
Collaborator

Signed-off-by: Christian Hartmann chris-hartmann@gmx.de

@Chartman123 Chartman123 added php PHP related ticket 2. developing Work in progress technical debt Technical issue labels May 2, 2026
@Chartman123 Chartman123 self-assigned this May 2, 2026
@Chartman123 Chartman123 added this to the 5.3 milestone May 2, 2026
@Chartman123 Chartman123 force-pushed the chore/appconfig branch 3 times, most recently from e671002 to a753942 Compare May 4, 2026 09:43
@codecov
Copy link
Copy Markdown

codecov Bot commented May 4, 2026

Codecov Report

❌ Patch coverage is 45.00000% with 11 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/Controller/ConfigController.php 30.76% 9 Missing ⚠️
lib/Migration/Version0010Date20190000000007.php 0.00% 1 Missing ⚠️
lib/Migration/Version010200Date20200323141300.php 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Chartman123 Chartman123 force-pushed the chore/appconfig branch 15 times, most recently from f0e5825 to 0d32035 Compare May 4, 2026 13:35
@Chartman123 Chartman123 marked this pull request as ready for review May 4, 2026 13:48
@Chartman123 Chartman123 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels May 4, 2026
@Chartman123 Chartman123 requested review from pringelmann and susnux May 4, 2026 13:48
…afety and clarity

Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
@Chartman123 Chartman123 force-pushed the chore/appconfig branch 2 times, most recently from 01fda31 to 750a215 Compare May 4, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews php PHP related ticket technical debt Technical issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant